Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.96%. Comparing base (f45e3ff) to head (a75f303).
⚠️ Report is 1 commits behind head on master.

@@           Coverage Diff           @@
##           master     #248   +/-   ##
=======================================
  Coverage   97.96%   97.96%           
=======================================
  Files           9        9           
  Lines         491      491           
  Branches       83       83           
=======================================
  Hits          481      481           
  Misses          6        6           
  Partials        4        4           

Should macOS-13 (the latest macOS GH runner with x86 arch) even be included at all?

Google's yamlfmt requires golang to be installed, so I chose this, which doesn't require extra dependencies

@wRAR @Gallaecio should I set indentation to 2 instead of the tool's default 4?

Yes, I think the closer to the original formatting, the better.

I also would like not to have the --- stuff at the beginning, specially because I have seen that fail on YAML readers that do not expect or support it, but I don’t see an option to turn that off in that hook :(

Given how more complete Google’s options look, I wonder if getting go installed in the CI would not be worth it. No strong opinion, though, I get it is weird to start asking every w3lib contributor to install Go.

I don't have preferences for a yaml formatting style so anything works for me :)

OK, lets merge as it is. Assuming the --- do not break anything, we can adjust things further in the future if someone feels strongly enough about it.